Skip to content

[FIX] postId는 postID, leaderId는 leaderId를 반환하도록 함#188

Merged
88guri merged 2 commits intodevelopfrom
fix/#187/leader-user-id
Jan 22, 2026
Merged

[FIX] postId는 postID, leaderId는 leaderId를 반환하도록 함#188
88guri merged 2 commits intodevelopfrom
fix/#187/leader-user-id

Conversation

@88guri
Copy link
Copy Markdown
Collaborator

@88guri 88guri commented Jan 22, 2026 •
edited by coderabbitai bot
Loading

📌 관련 이슈

✨ 변경 사항

  • leaderUserID가 postId를 반환하고있어서 수정했습니다

📸 테스트 증명 (필수)

📚 리뷰어 참고 사항

✅ 체크리스트

  • 브랜치 전략(git flow)을 따랐나요? (develop -> feat/...)
  • 로컬에서 빌드 및 실행이 정상적으로 되나요?
  • 불필요한 주석(TODO 등)이나 더미 코드는 제거했나요?
  • 컨벤션(커밋 메시지, 코드 스타일)을 지켰나요?

Summary by CodeRabbit

릴리스 노트

  • 개선 사항
    • 배송 확인 응답에 그룹 구매 게시물 리더 정보가 포함되도록 개선되었습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@88guri 88guri requested a review from PBEM22 January 22, 2026 19:24
@88guri 88guri self-assigned this Jan 22, 2026
@88guri 88guri added 🚨 Fix 버그 수정 🐵 시현 시현이에요 ✋Participation 공구 참여, 관리 labels Jan 22, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 22, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

confirmDelivered 메서드에서 그룹 바이 포스트의 리더 ID를 조회하여 LeaderUserIdResponse 반환값에 추가하는 변경사항입니다. 응답 객체의 생성자 시그니처가 확장되어 postId와 leaderUserId를 모두 반환합니다.

Changes

Cohort / File(s) Summary
응답 DTO 및 서비스 로직
src/main/java/org/sopt/poti/domain/participation/dto/response/LeaderUserIdResponse.java, src/main/java/org/sopt/poti/domain/participation/service/ParticipationService.java		 confirmDelivered 메서드에서 order.getGroupBuyPost().getLeader().getId()를 통해 leaderUserId를 조회하여 응답에 포함. LeaderUserIdResponse 생성자 시그니처 변경: (Long postId) → (Long postId, Long leaderUserId)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 주요 변경사항을 명확히 반영하고 있습니다. postId와 leaderId가 올바르게 반환되도록 수정했다는 내용이 직관적으로 전달됩니다.
Linked Issues check ✅ Passed PR 변경사항이 #187 이슈의 요구사항을 충족합니다. leaderId 대신 postId를 반환하던 버그를 수정하여 LeaderUserIdResponse에 leaderUserId 필드를 추가했습니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 #187 이슈 해결을 위한 범위 내입니다. LeaderUserIdResponse 생성자 수정과 confirmDelivered 메서드의 leaderUserId 추가 로직만 포함되어 있습니다.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

coderabbitai[bot]
coderabbitai bot reviewed Jan 22, 2026
View reviewed changes
Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
src/main/java/org/sopt/poti/domain/participation/service/ParticipationService.java (1)

75-89: LeaderUserIdResponse 생성자 인자 누락으로 컴파일/반환값 오류

현재 new LeaderUserIdResponse(leaderUserId)로 호출하고 있어, 생성자가 (postId, leaderUserId)로 변경된 경우 컴파일 실패 또는 의도한 postId 누락이 발생합니다. PR 목적에 맞게 postId도 함께 반환하도록 수정해주세요.

🐛 수정 제안 
-    return new LeaderUserIdResponse(leaderUserId);
+    return new LeaderUserIdResponse(postId, leaderUserId);

@88guri 88guri merged commit ac01240 into develop Jan 22, 2026
2 checks passed
@coderabbitai coderabbitai bot mentioned this pull request Jan 22, 2026
Merged
4 tasks
@88guri 88guri deleted the fix/#187/leader-user-id branch January 23, 2026 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

@PBEM22 PBEM22 PBEM22 approved these changes

Assignees

@88guri 88guri

Labels

🚨 Fix 버그 수정 ✋Participation 공구 참여, 관리 🐵 시현 시현이에요

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[FIX] leaderId 반환 대신 postId 반환중

2 participants

@88guri @PBEM22